Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lang German added #165

Closed
wants to merge 1 commit into from
Closed

Conversation

Laurin-migi
Copy link

I come from Switzerland and have therefore added a German language to your great package.

@abjerner
Copy link
Member

abjerner commented Feb 1, 2023

Thanks a lot @Laurin-migi 👍

Any reason why this PR is targeting v2/main (Umbraco 8) and not v4/main (Umbraco 10+)?

@Laurin-migi
Copy link
Author

Laurin-migi commented Feb 2, 2023

Any reason why this PR is targeting v2/main (Umbraco 8) and not v4/main (Umbraco 10+)?

yes, thats because we use umbraco v8 and we will use umvraco 10 later. When I have time, I will make a new PR with v10

Copy link
Member

@abjerner abjerner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Laurin-migi

I've found a minor issue. Even for the German translation, this is supposed to be the English word Content, right?

If you can look into this, I'd be happy to merge, and then push a new release for Umbraco 8 😉

<key alias="reload">Neu laden</key>
<key alias="edit">Bearbeiten</key>
<key alias="delete">Löschen</key>
<key alias="content">Contet</key>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be Content like line 40?

@limbobot limbobot added umbraco/v8 Issues and tasks related to Umbraco 8. status/stale Marked as stale due to inactivity labels Apr 15, 2023
@limbobot
Copy link

Hi @Laurin-migi

Thanks so much for submitting this pull request and helping to improve the package! We had asked for some additional information, but since we haven’t heard back from you, we’ll go ahead and close this PR for now to keep things organized.

If you’d like to revisit it, don’t hesitate to reopen the PR or create a new one when you’re ready. We’re grateful for your contribution and hope to see more from you in the future. 😊

@limbobot limbobot closed this Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community/pr state/needs-more-info status/stale Marked as stale due to inactivity umbraco/v8 Issues and tasks related to Umbraco 8.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants